Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat] Add s3 uploading from sdk #31

Merged
merged 13 commits into from
Aug 20, 2024
Merged

Conversation

jiyuu-jin
Copy link
Collaborator

@jiyuu-jin jiyuu-jin commented Jul 1, 2024

Description

Please note this is just an initial implementation to get it working with the shared browser code, it's gonig to need some follow-up cleanup and improvement.

Adds s3 uploading to CLI via @valist/[email protected] implementation. Closes https://github.com/HyperPlay-Gaming/product-management/issues/446

Also closes #29 by skipping publishing and for the time being ignores the keys logic to be tested in another branch.

Changes

  • Split loginAndPublish function into login and publish functions
  • Added baseURL directly to axios client
  • Refactored uploadRelease to use an axiosInstance with cookie jar
  • Removed all IPFS functions and references.
  • Added new test mocks for s3 upload requests and logic.

@jiyuu-jin jiyuu-jin self-assigned this Jul 11, 2024
@jiyuu-jin jiyuu-jin added the enhancement New feature or request label Jul 11, 2024
Copy link
Contributor

@BrettCleary BrettCleary left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upload is working for me. I did not test download. can't find anything wrong with the code

maybe we should just remove the view the release at: line since the url will not work since it isn't a presigned url.

Upload progress for windows_amd64: 100
Successfully uploaded files to HyperPlay: https://gateway-b3.valist.io/hyperplaycd/irontest/v0.0.7
Publishing release... done
Confirming transaction 0xfef073aca4877f0eb4cef9b55d325cdc164e0aadba8fb176aa0e80fb823858ea... done
Successfully published hyperplaycd/irontest/v0.0.7!
view the release at:
https://gateway-b3.valist.io/hyperplaycd/irontest/v0.0.7

Copy link

@red-game-dev red-game-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀 reviewed only the code, I think it should be tested still 🙌

@jiyuu-jin jiyuu-jin merged commit 4290d22 into main Aug 20, 2024
1 check passed
@BrettCleary BrettCleary deleted the feat/add-s3-uploading-from-sdk branch August 30, 2024 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants